Skip to content

modules/lsp/server: declare package defaults #3243

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Apr 30, 2025

Conversation

MattSturgeon
Copy link
Member

@MattSturgeon MattSturgeon commented Apr 29, 2025

Initial design for declaring per-server lsp.servers.<name>.package defaults.

  • Change lsp.servers to a freeform submodule
  • Call the server.nix module using lib.importApply
    • This allows passing in server-specific info like name and package
  • Declare a lsp.servers.<name> option for each packaged server

@MattSturgeon

This comment was marked as resolved.

@MattSturgeon MattSturgeon changed the title wip: modules/lsp: set per-server package defaults modules/lsp/server: declare package defaults Apr 29, 2025
@MattSturgeon MattSturgeon marked this pull request as ready for review April 29, 2025 21:59
@MattSturgeon MattSturgeon requested review from GaetanLepage and a team April 29, 2025 22:01
Copy link
Member

@GaetanLepage GaetanLepage left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, well done @MattSturgeon !

Convert the `attrsOf (servers.nix)` option to a freeform submodule.

Declare a `servers.nix` option for each lsp server listed in
`lsp-packages.nix` that has a known nixpkgs package.
@MattSturgeon

This comment was marked as resolved.

This comment was marked as resolved.

@mergify mergify bot merged commit e34eaf8 into nix-community:main Apr 30, 2025
4 checks passed
Copy link
Contributor

mergify bot commented Apr 30, 2025

This pull request, with head sha e34eaf8395855bf8b1b1a40b38f4c9e9abe2a38d, has been successfully merged with fast-forward by Mergify.

This pull request will be automatically closed by GitHub.

As soon as GitHub detects that the sha e34eaf8395855bf8b1b1a40b38f4c9e9abe2a38d is part of the main branch, it will mark this pull request as merged.

It is possible for this pull request to remain open if this detection does not happen, this usually happens when a force-push is done on this branch lsp-packages, this means GitHub will fail to detect the merge.

@mergify mergify bot temporarily deployed to github-pages April 30, 2025 16:02 Inactive
@MattSturgeon MattSturgeon deleted the lsp-packages branch April 30, 2025 16:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants